Skip to content

Preserved width/height on scaleUp/Down a Rectangle #2047

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

akoch-yatta
Copy link
Contributor

This commit extends the logic when a Rectangle is scaled up/down from points to pixels regarding width and height.

Fixes #2003

Copy link
Contributor

github-actions bot commented Apr 22, 2025

Test Results

   545 files     545 suites   26m 26s ⏱️
 4 373 tests  4 355 ✅  18 💤 0 ❌
16 634 runs  16 496 ✅ 138 💤 0 ❌

Results for commit 75b80d7.

♻️ This comment has been updated with latest results.

@akoch-yatta akoch-yatta force-pushed the Fix-scaling-of-rectangles branch from 742add3 to 57aaf23 Compare April 22, 2025 15:29
Copy link
Contributor

@HeikoKlare HeikoKlare left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change is sound and properly solves #2003

I couldn't find any unexpected side effects so far, so let's give this a try.

@HeikoKlare HeikoKlare force-pushed the Fix-scaling-of-rectangles branch from 57aaf23 to 50376ac Compare April 23, 2025 07:24
This commit extends the logic when a Rectangle is scaled up/down from points
to pixels regarding width and height.

Fixes eclipse-platform#2003
@HeikoKlare HeikoKlare force-pushed the Fix-scaling-of-rectangles branch from 50376ac to 75b80d7 Compare April 23, 2025 07:38
@HeikoKlare
Copy link
Contributor

Merging this despite Jenkins failure caused by https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/issues/5892

Since master builds seem to still work on Jenkins, the subsequent master build should hopefully work, thus I will check results of that build to retrospectively validate this PR.

@HeikoKlare HeikoKlare merged commit d4caabf into eclipse-platform:master Apr 23, 2025
10 of 12 checks passed
@HeikoKlare HeikoKlare deleted the Fix-scaling-of-rectangles branch April 23, 2025 08:13
@HeikoKlare
Copy link
Contributor

No new issues on subsequent master build for d4caabf

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Links in empty Package Explorer are cut off Links in empty Package Explorer are cut off
2 participants